Skip to content

iOS: Add early resolution and conditional streaming for GET requests#108

Draft
Copilot wants to merge 48 commits intomasterfrom
copilot/migrate-ios-implementation-to-alamofire
Draft

iOS: Add early resolution and conditional streaming for GET requests#108
Copilot wants to merge 48 commits intomasterfrom
copilot/migrate-ios-implementation-to-alamofire

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 29, 2026 •
edited
Loading

Fix Parallel Request Cancellation Issue

Problem:

When running multiple requests in parallel, they were getting cancelled during session recreation. This happened because recreateSession() was cancelling all active requests.

Checklist:

  • Remove Request Cancellation:

    • Removed code that cancels active requests in recreateSession()
    • Active requests now continue on the old session until they complete
    • Keep old session alive for 30 seconds to allow requests to finish

  • Test:

    • Verify parallel requests work without cancellation

Changes:

Before:

  • recreateSession() cancelled all active requests when creating a new session
  • Old session was kept alive for only 0.5 seconds
  • Parallel requests would fail with cancellation errors

After:

  • recreateSession() does NOT cancel active requests
  • Active requests continue running on the old session
  • Old session is kept alive for 30 seconds to allow requests to complete
  • New requests use the new session with updated event monitors/interceptors
  • Parallel requests can now run without interference

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

Copilot AI and others added 5 commits March 29, 2026 16:04
@farfromrefug
Add Alamofire Swift wrappers and update TypeScript to use them
0ec2ac6 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/fc9101f9-5596-43a3-ab69-ff9c48eeef06

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Fix parameter encoding in Swift wrapper and add error key constants
2a50a0b 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/fc9101f9-5596-43a3-ab69-ff9c48eeef06

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Fix request chaining in multipart POST and add migration documentation
64999ba 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/fc9101f9-5596-43a3-ab69-ff9c48eeef06

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Address code review feedback: fix task casting, security validation, …
264dd58 
…and deprecated APIs

Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/fc9101f9-5596-43a3-ab69-ff9c48eeef06

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Add migration summary documentation
814d31f 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/fc9101f9-5596-43a3-ab69-ff9c48eeef06

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate iOS implementation from AFNetworking to Alamofire Migrate iOS implementation from AFNetworking to Alamofire Mar 29, 2026
Copilot AI requested a review from farfromrefug March 29, 2026 16:13
Copilot AI and others added 4 commits March 29, 2026 16:26
@farfromrefug
Implement cleaner API names and streaming downloads for iOS
1e2e3f7 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/99200ca0-c02e-437d-bbdc-d3c5f6f221c3

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Add comprehensive documentation for API improvements and streaming do…
59c4261 
…wnloads

Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/99200ca0-c02e-437d-bbdc-d3c5f6f221c3

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Fix code review issues: safe unwrapping and documentation numbering
d29ca73 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/99200ca0-c02e-437d-bbdc-d3c5f6f221c3

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Add comprehensive API improvements summary documentation
9d81a53 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/99200ca0-c02e-437d-bbdc-d3c5f6f221c3

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
Copilot AI changed the title Migrate iOS implementation from AFNetworking to Alamofire Improve iOS API with clean method names and memory-efficient streaming downloads Mar 29, 2026
Copilot AI and others added 2 commits March 29, 2026 17:05
Copilot AI changed the title Improve iOS API with clean method names and memory-efficient streaming downloads Align iOS response handling with Android: load data in memory for post-request processing Mar 29, 2026
Copilot AI and others added 4 commits March 29, 2026 19:25
@farfromrefug
Implement streaming downloads for iOS GET requests
98bb864 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/07e4da87-be33-46f4-872f-e397b5e6c049

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Update documentation for streaming download behavior
b4e1094 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/07e4da87-be33-46f4-872f-e397b5e6c049

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Add comprehensive iOS streaming implementation documentation
fa937d1 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/07e4da87-be33-46f4-872f-e397b5e6c049

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Add flow diagrams for iOS streaming implementation
07806a8 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/07e4da87-be33-46f4-872f-e397b5e6c049

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
Copilot AI changed the title Align iOS response handling with Android: load data in memory for post-request processing Implement streaming downloads for iOS GET requests using temp file approach Mar 29, 2026
Copilot AI and others added 3 commits April 12, 2026 15:19
Copilot AI and others added 4 commits April 12, 2026 16:33
@farfromrefug
Update AlamofireWrapper upload/download methods to use request IDs an…
1483106 
…d NSHTTPURLResponse

Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/cb3011ae-c8b8-4453-9b0e-af75b9bb74ce

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Update TypeScript typings and remove responseSerializer.deserialize c…
02b44e4 
…alls

Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/cb3011ae-c8b8-4453-9b0e-af75b9bb74ce

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Remove useLegacy parameter and update upload methods in request.ios.ts
d4f705d 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/cb3011ae-c8b8-4453-9b0e-af75b9bb74ce

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
@farfromrefug
Update downloadToTemp calls to use new signatures
61235bb 
Agent-Logs-Url: https://github.com/nativescript-community/https/sessions/cb3011ae-c8b8-4453-9b0e-af75b9bb74ce

Co-authored-by: farfromrefug <655344+farfromrefug@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@farfromrefug farfromrefug Awaiting requested review from farfromrefug

Assignees

@farfromrefug farfromrefug

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@farfromrefug